Skip to content

Conversation

caspervonb
Copy link
Collaborator

@caspervonb caspervonb commented Oct 7, 2025

Brings install time of nats-server from 2-3 minutes to 2-3 seconds.

@caspervonb caspervonb force-pushed the use-server-binaries-from-nats-dev branch 6 times, most recently from c49a18e to 430e3fd Compare October 7, 2025 10:30
@caspervonb caspervonb requested a review from Copilot October 7, 2025 10:31
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces a reusable GitHub Action to optimize NATS server installation in CI workflows, reducing installation time from 2-3 minutes to 2-3 seconds through caching.

  • Replaces manual installation scripts with a centralized GitHub Action
  • Implements caching strategy for NATS server binaries across workflow runs
  • Simplifies workflow configuration by eliminating manual PATH management

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
.github/actions/setup-nats-server/action.yml New composite action that downloads, caches, and configures NATS server
.github/workflows/test.yml Updated to use the new action and simplified matrix configuration
nats/scripts/install_nats.sh Removed obsolete installation script
nats/tests/utils.py Updated comment to reference official NATS installation docs

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@caspervonb caspervonb force-pushed the use-server-binaries-from-nats-dev branch 7 times, most recently from 6f2ee8a to a467549 Compare October 7, 2025 10:49
@caspervonb caspervonb force-pushed the use-server-binaries-from-nats-dev branch 2 times, most recently from 21e6cd5 to 3e13e9a Compare October 13, 2025 17:54
@caspervonb
Copy link
Collaborator Author

Windows being stubborn, going a different route.

@caspervonb caspervonb closed this Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant